Skip to content

Ensure code format #2138

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 7 commits into from
Aug 9, 2023
Merged

Ensure code format #2138

merged 7 commits into from
Aug 9, 2023

Conversation

trusktr
Copy link
Member

@trusktr trusktr commented Jul 17, 2023

Summary

We're using prettier, so ensure we check code format as part of the lint script. Also add a prettier script for manually formatting the whole code base.

The issue and pull request templates needed formatting, so I went ahead and made some updates at the same time. This PR is now using the updated PR template.

Related issue, if any:

Part of

Continues from PR

What kind of change does this PR introduce?

Build-related changes
Repo settings

For any code change,

  • Related documentation has been updated, if needed
  • Related tests have been added or updated, if needed

Does this PR introduce a breaking change?

No

Tested in the following browsers:

  • Chrome
  • Firefox
  • Safari
  • Edge

trusktr added 3 commits July 16, 2023 10:54
…private methods and properties to start to encapsulate Docsify

Also some small tweaks:

- move initGlobalAPI out of Docsify.js to start to encapsulate Docsify
- move ajax to utils folder
- fix some type definitions and improve content in some JSDoc comments
- use concise class field syntax
- consolidate duplicate docsify-ignore comment removal code

This handles a task in [Simplify and modernize Docsify](#2104), as well as works towards [Encapsulating Docsify](#2135).
…rettier script for manually formatting the whole code base
@trusktr trusktr requested a review from a team July 17, 2023 05:58
@vercel
Copy link

vercel bot commented Jul 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docsify-preview ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 9, 2023 9:48am

Koooooo-7
Koooooo-7 previously approved these changes Jul 17, 2023
Copy link
Member

@Koooooo-7 Koooooo-7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Koooooo-7 Koooooo-7 requested a review from a team July 17, 2023 11:20
@Koooooo-7 Koooooo-7 changed the base branch from move-private-functions-properties-into-classes to develop August 9, 2023 03:33
@Koooooo-7 Koooooo-7 dismissed their stale review August 9, 2023 03:33

The base branch was changed.

Koooooo-7
Koooooo-7 previously approved these changes Aug 9, 2023
@Koooooo-7 Koooooo-7 merged commit 7bbf13d into develop Aug 9, 2023
@Koooooo-7 Koooooo-7 deleted the ensure-code-format branch August 9, 2023 09:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants